Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add version information #177

Merged
merged 18 commits into from
Jun 14, 2024
Merged

Add version information #177

merged 18 commits into from
Jun 14, 2024

Conversation

hikinggrass
Copy link
Contributor

This outputs version information in the manager on startup and when passed a --version parameter on the command line

@hikinggrass hikinggrass force-pushed the feature/add-version-information branch from 97866c2 to 7b2f795 Compare April 5, 2024 10:31
@hikinggrass hikinggrass force-pushed the feature/add-version-information branch 2 times, most recently from 6df8547 to f359187 Compare April 22, 2024 10:35
mhei and others added 13 commits June 10, 2024 18:04
This adds the Unix/Linux traditional commmand line argument "--version"
so that it is possible to retrieve the version of the binary.

Example output:
/usr/bin/manager (everest-framework 0.10.1)

Signed-off-by: Michael Heimpold <michael.heimpold@chargebyte.com>
The generated code of each C++ module uses the module loader for
processing command line parameters etc.
This is why we have to extend the class and pass the version information
from the generated files into the framework library here.

Signed-off-by: Michael Heimpold <michael.heimpold@chargebyte.com>
change project_name and project_version paramters to a struct VersionInformation that also includes a git_version member

Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
It is now automatically generated in ${CMAKE_CURRENT_BINARY_DIR}/generated/include/version_information.hpp

Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
This file can be generated during a build of everest-core

Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
generate version info header directly

Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
This has no version information (for now)

Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
@hikinggrass hikinggrass force-pushed the feature/add-version-information branch from f359187 to 0a415c2 Compare June 13, 2024 10:06
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
@hikinggrass hikinggrass marked this pull request as ready for review June 13, 2024 10:09
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Copy link
Contributor

@andistorm andistorm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…nformation

# Conflicts:
#	CMakeLists.txt

Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
Signed-off-by: Kai-Uwe Hermann <kai-uwe.hermann@pionix.de>
@hikinggrass hikinggrass merged commit 5836534 into main Jun 14, 2024
4 of 5 checks passed
@hikinggrass hikinggrass deleted the feature/add-version-information branch June 14, 2024 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants